Skip to content

Pause iterative refinement on ACP timeout instead of failing the pipeline#177

Merged
Jiaaqiliu merged 1 commit intoaiming-lab:mainfrom
CKwin26:ck/stage13-paused
Mar 30, 2026
Merged

Pause iterative refinement on ACP timeout instead of failing the pipeline#177
Jiaaqiliu merged 1 commit intoaiming-lab:mainfrom
CKwin26:ck/stage13-paused

Conversation

@CKwin26
Copy link
Copy Markdown

@CKwin26 CKwin26 commented Mar 29, 2026

Summary

  • pause Stage 13 iterative refinement when ACP prompt generation times out
  • keep stage metadata resumable by pointing non-done stages back to themselves
  • surface paused runs in the CLI and pipeline summary/runner output

Testing

  • python -m pytest tests/test_rc_cli.py tests/test_rc_runner.py tests/test_rc_executor.py -k "paused or timeout"

Notes

This change is intentionally scoped to timeout pause/resume behavior and does not include the larger prompt-compaction or real-data-guard experiments from my local branch.

@Jiaaqiliu Jiaaqiliu merged commit a4fda10 into aiming-lab:main Mar 30, 2026
Jiaaqiliu added a commit that referenced this pull request Mar 30, 2026
…tput

PR #177 introduced Cyrillic 'бк' (U+0431 U+043A) instead of dashes in the
PAUSED status print line due to an encoding issue.
Jiaaqiliu added a commit that referenced this pull request Mar 30, 2026
- Remove stale _DOMAIN_KEYWORDS import from executor.py (PR #180 removed it)
- Fix test_paper_draft_blocked_with_no_metrics for new domain detector
- Replace Cyrillic characters with ASCII dashes in PAUSED stage output (PR #177)
- Add pytest-asyncio to dev dependencies for CI compatibility (PR #185)
- Guard os.getuid()/os.getgid() for Windows in figure renderer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants